Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Clarify documentation for only_for_reauth #16737

Merged
merged 3 commits into from
Dec 7, 2023
Merged

Clarify documentation for only_for_reauth #16737

merged 3 commits into from
Dec 7, 2023

Conversation

dklimpel
Copy link
Contributor

@dklimpel dklimpel commented Dec 7, 2023

Part of #14686

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Pull request includes a sign off
  • Code style is correct
    (run the linters)

Signed-off-by: Dirk Klimpel [email protected]

@dklimpel dklimpel requested a review from a team as a code owner December 7, 2023 17:38
DMRobertson
DMRobertson approved these changes Dec 7, 2023
View reviewed changes
Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

changelog.d/16737.doc Outdated Show resolved Hide resolved
@DMRobertson DMRobertson merged commit aa983c7 into matrix-org:develop Dec 7, 2023
31 checks passed
@dklimpel dklimpel deleted the fix_doc_reauth branch December 7, 2023 18:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DMRobertson DMRobertson DMRobertson approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dklimpel @DMRobertson